Skip to content

Document pip constraints CI enforcement for all dependencies #198

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

booxter
Copy link
Contributor

@booxter booxter commented May 14, 2025

This work is being implemented in the main (instructlab/instructlab)
repository. The same approach should be applied across all the repos.

Signed-off-by: Ihar Hrachyshka [email protected]

@booxter booxter force-pushed the ci-deps-constraints branch 2 times, most recently from 2a8afa5 to 2e914bc Compare May 14, 2025 16:25
@booxter booxter force-pushed the ci-deps-constraints branch 2 times, most recently from dd48e12 to fe899a4 Compare May 15, 2025 13:26
@booxter booxter force-pushed the ci-deps-constraints branch from fe899a4 to 7c318aa Compare May 22, 2025 16:15
This work is being implemented in the main (instructlab/instructlab)
repository. The same approach should be applied across all the repos.

Signed-off-by: Ihar Hrachyshka <[email protected]>
@booxter booxter force-pushed the ci-deps-constraints branch from 7c318aa to a7e6ce4 Compare May 28, 2025 23:42
@booxter
Copy link
Contributor Author

booxter commented May 28, 2025

Updated a bit to reflect that we now have a new ci-actions action for periodic updates.

Copy link
Contributor

@ktdreyer ktdreyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're implementing this now, so I think we should merge this as a record.

We can apply further changes in future PRs if we find we need that.

@booxter
Copy link
Contributor Author

booxter commented Jun 4, 2025

I'm self-merging because the document is effectively implemented across repos and no longer in the decision making phase.

@booxter booxter merged commit 30cb9d6 into instructlab:main Jun 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants